Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add cookie implementation details for the NPM package #7051

Merged
merged 3 commits into from
Dec 18, 2024
Merged

Add cookie implementation details for the NPM package #7051

merged 3 commits into from
Dec 18, 2024

Conversation

niallzato
Copy link
Contributor

Proposed changes

Add cookie implementation details for the NPM package.

@niallzato niallzato added the KCS label Sep 19, 2024
@niallzato niallzato requested a review from a team as a code owner September 19, 2024 15:47
@niallzato
Copy link
Contributor Author

Hi @stayseesong apologies missed the change request, and it's no longer appearing. Good to approve/merge?

@niallzato
Copy link
Contributor Author

@stayseesong good to merge?

@niallzato
Copy link
Contributor Author

@stayseesong or @forstisabella are we good to merge this? Thanks!

@forstisabella forstisabella merged commit 6735158 into develop Dec 18, 2024
4 checks passed
@forstisabella forstisabella deleted the niall/cookie_npm branch December 18, 2024 23:34
@github-actions GitHub Actions
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@forstisabella forstisabella forstisabella approved these changes

@stayseesong stayseesong stayseesong approved these changes

Assignees
No one assigned
Labels
KCS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@niallzato @stayseesong @forstisabella